Skip to content

Conversation

@ankuns
Copy link
Contributor

@ankuns ankuns commented May 5, 2025

This commit brings in zephyr containing the fix of tx-en-settle-time-us default value for the nrf21540 Front-End Module.

The doc commit is cherry picked from NCS 3.0 branch 61bd2af

Ref: KRKNWK-20212

@ankuns ankuns requested review from a team as code owners May 5, 2025 07:28
@github-actions github-actions bot added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels May 5, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 5, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@464150f nrfconnect/sdk-zephyr@893c333 (main) nrfconnect/[email protected]

All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@ankuns ankuns requested a review from a team as a code owner May 5, 2025 07:30
@github-actions github-actions bot added the doc-required PR must not be merged without tech writer approval. label May 5, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 5, 2025

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 4

Inputs:

Sources:

sdk-nrf: PR head: f6bd3a4401504f88069dc57ab1ca8cf8acf6b716
zephyr: PR head: 893c3332454a29c1611fa90aa8ceaf87a6bdf848

more details

sdk-nrf:

PR head: f6bd3a4401504f88069dc57ab1ca8cf8acf6b716
merge base: 7ba47ec38c70d26a0e99a1df7a18992123525d84
target head (main): 7ba47ec38c70d26a0e99a1df7a18992123525d84
Diff

zephyr:

PR head: 893c3332454a29c1611fa90aa8ceaf87a6bdf848
merge base: 464150f23f1f19d427145cb8e3d2a47f4d15de11
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (3)
doc
│  ├── nrf
│  │  ├── app_dev
│  │  │  ├── device_guides
│  │  │  │  ├── fem
│  │  │  │  │  │ fem_nrf21540_gpio.rst
west.yml
zephyr
│  ├── dts
│  │  ├── bindings
│  │  │  ├── net
│  │  │  │  ├── wireless
│  │  │  │  │  │ nordic,nrf21540-fem.yaml

Outputs:

Toolchain

Version:
Build docker image:

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain
  • ◻️ Build twister
  • ◻️ Integration tests
    • ◻️ test-fw-nrfconnect-rs
    • ◻️ test-fw-nrfconnect-fem
    • ◻️ test-low-level
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-boot
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_mosh
    • test-fw-nrfconnect-nrf-iot_positioning
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-sdk-audio
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

@github-actions
Copy link

github-actions bot commented May 5, 2025

You can find the documentation preview for this PR here.

Preview links for modified nRF Connect SDK documents:

https://ncsdoc.z6.web.core.windows.net/PR-22136/nrf/app_dev/device_guides/fem/fem_nrf21540_gpio.html

@ankuns ankuns force-pushed the nrf21540_fix_default_tx_en_settle_time_cp branch from f5bcf77 to 470dbc8 Compare May 6, 2025 06:38
ankuns added 2 commits May 6, 2025 08:39
This commit brings in zephyr containing the fix of
tx-en-settle-time-us default value for the nrf21540 Front-End Module.

Signed-off-by: Andrzej Kuros <[email protected]>
The default timing value for the `tx-en-settle-time-us` property
is changed to 26. For the nrf54L the time is 1us bigger.
This commit adjusts fem doc recommendation for the nRF54L.

Signed-off-by: Andrzej Kuros <[email protected]>
@ankuns ankuns force-pushed the nrf21540_fix_default_tx_en_settle_time_cp branch from 470dbc8 to f6bd3a4 Compare May 6, 2025 06:39
@NordicBuilder NordicBuilder removed the DNM label May 6, 2025
@rlubos rlubos merged commit 66234d1 into nrfconnect:main May 6, 2025
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. doc-required PR must not be merged without tech writer approval. manifest manifest-zephyr

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants